Skip to content

Rename DefaultPolicies to ConiksPolicies#41

Merged
vqhuy merged 1 commit into
masterfrom
policy
Jul 28, 2016
Merged

Rename DefaultPolicies to ConiksPolicies#41
vqhuy merged 1 commit into
masterfrom
policy

Conversation

@vqhuy
Copy link
Copy Markdown
Member

@vqhuy vqhuy commented Jul 28, 2016

As discussed in #6

Comment thread merkletree/policy.go Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: Conisk should be Coniks.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops :(

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem!

@masomel
Copy link
Copy Markdown
Member

masomel commented Jul 28, 2016

I think this is ready to be merged. The additional mandatory policy fields will be added as part of #35 (as we decided in #37 (comment)), right?

@vqhuy
Copy link
Copy Markdown
Member Author

vqhuy commented Jul 28, 2016

The additional mandatory policy fields will be added as part of #35 (as we decided in #37 (comment)), right?

Right.

@vqhuy vqhuy merged commit a461381 into master Jul 28, 2016
@vqhuy vqhuy deleted the policy branch July 28, 2016 18:45
@masomel masomel mentioned this pull request Sep 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants